Skip to content

Update ACS URL format in SSO instructions#349

Merged
jtaylor-cs merged 2 commits intomainfrom
single-sign-on-with-google-g-suite-jtaylor
Mar 20, 2026
Merged

Update ACS URL format in SSO instructions#349
jtaylor-cs merged 2 commits intomainfrom
single-sign-on-with-google-g-suite-jtaylor

Conversation

@jtaylor-cs
Copy link
Contributor

URL was incorrect and contradictory to the screenshot

URL was incorrect and contradictory to the screenshot
Copilot AI review requested due to automatic review settings February 20, 2026 16:49
@vercel
Copy link

vercel bot commented Feb 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cloudsmith-docs Ready Ready Preview, Comment Mar 20, 2026 1:27pm

Request Review

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Google (G-Suite) SSO setup guide to use the correct Cloudsmith SAML ACS/Entity ID URL format, aligning the written instructions with the expected URL structure.

Changes:

  • Corrected the SAML ACS URL / Entity ID example to include the /orgs/ path segment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@colinmoynes colinmoynes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few conversations open around grammar and naming.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jtaylor-cs
Copy link
Contributor Author

I did not update MY_ORG_NAME to WORKSPACE as I do not want to create a conflict with the current image, which uses MY_ORG_NAME

@jtaylor-cs jtaylor-cs removed the request for review from paulmay-cloudsmith March 20, 2026 13:34
@jtaylor-cs jtaylor-cs merged commit cc0f1cf into main Mar 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants